-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update to node20 #209
update to node20 #209
Conversation
Node20 requires glibc 2.28. CentOS 7 ships with glibc 2.17. Therefore, once we upgrade CentOS 7 will become end-of-life on June 30, 2024. Therefore, I think that we should wait until June 30 to upgrade CentOS 8 ships with glibc 2.28. Therefore, self-hosted CentOS 8 runners will be able to run Node20. |
@DilumAluthge Why not just release a new version? That's what setup-python did, for example. CentOS 7 users will stay on v1 and everyone else will move on |
If node16 is EOL, shouldn't we move quickly to node20? Will those nodes stop working at some point? |
All the other major actions bumped their major version to handle this. That seems like a good solution here too? Just make it |
Because then all bugfixes have to be backported to the old version |
I haven't seen any announcement that GitHub-hosted runners will lose the ability to run |
If node20 is faster, wouldn't it be nice for people who don't worry about CentOS 7 etc to be able to use it? v2 seems like a good choice. EDIT: I see your point about backporting. What really depends on CentOS 7 in our ecosystem? I thought pretty much most package CI scripts use ubuntu-latest. I understand June 30 is not that far away... |
Personally, I'm not pleased that GitHub is forcing the upgrade a month and a half earlier than the CentOS 7 EOL. But unfortunately there's nothing that we can do. So May 13 is when we'll need to switch to Node20. It'll probably be good to give ourselves a day or two of buffer, so let's plan on merging this PR on Monday, May 13, 2024 and making the new v2.0.0 release of this action on the same day. |
I think you'll want to release 2.0 before the runner cut off date to give those who use this action some time to update their workflows to use 2.0 before nodejs 16 is removed. Otherwise they're stuck with a broken 1.0 for a bit. |
Ah, that's fair. How much of a lead time do you think would make sense? Would one week be enough? Maybe two weeks? |
Honestly? About two months ago would have been nice. Now is second best. Just 1-2 weeks before is super short. Thousands of projects need to adapt to this change. Allowing for a smooth transition over multiple months like this is a major reason why GitHub actions can have multiple versions supported in parallel. |
Why not do it right away? What will we gain with CentOS 7 support for another 6 weeks? |
The version number bump will be done here: #228 |
Thanks @DilumAluthge! |
I wonder if we should add a deprecation notice to Otherwise I imagine lots of people will never get newer versions as they've just copied workflows that included Similarly, PkgTemplates.jl and other places that contain a CI template need to be updated. (There's a reason why in the past we've tagged a few releases as non-breaking even if they were technically breaking on enterprise/self-hosted setups) |
Or from this repo's README 😄 |
Fixes #208
Node 16 is deprecated, see https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/